#691 - Adopt wayfinder + implement, reshape planning pipeline into spec/plan review#693
Open
bracyw wants to merge 5 commits into
Open
#691 - Adopt wayfinder + implement, reshape planning pipeline into spec/plan review#693bracyw wants to merge 5 commits into
bracyw wants to merge 5 commits into
Conversation
…ec/plan review Take Matt Pocock's main flow as the spine (grill-with-docs -> to-spec -> to-tickets -> implement) with wayfinder as the exploration on-ramp that merges at to-spec (one map can feed several specs). Add the implement skill (drives tdd -> code-review -> commit; Argos /commit convention). Reuse the existing /code-review rather than vendoring Matt's. Reshape #669's branch planning pipeline into a narrow spec/plan review requirement: to-spec and to-tickets stage artifacts as files under docs/spec/<name>/ and review them as a PR before publishing, keeping unreviewed tickets off the tracker. Decoupled from grilling and wayfinder; restores the local-file draft mode dropped in #683. Add Wayfinding operations + ai-workflow / wayfinder:* labels to the tracker doc, glossary terms, CLAUDE.md flow wiring, and ADR-0005. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SdH2gxRtfjoqUWa5hG8SvF
This was referenced Jul 11, 2026
Add .claude/skills/code-review as Matt Pocock's Standards-vs-Spec review (parallel sub-agents + Fowler smell baseline), Argos-adapted: Standards axis reads the CLAUDE.md convention files, Spec axis reads the originating issue or a docs/spec/ spec. implement's /code-review step now resolves to this. Update ADR-0005 accordingly. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SdH2gxRtfjoqUWa5hG8SvF
…ariant to-spec now says its staged spec.md is temporary (deleted by to-tickets, or by the author for a spec-only effort). Wayfinder's handoff drops the straight-to-implement bypass: the target varies (to-spec for a spec, to-tickets for an in-place change) but build work always goes through a reviewed ticket, since to-spec and to-tickets both stage-and-review. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SdH2gxRtfjoqUWa5hG8SvF
Keep wayfinder verbatim-upstream except the required GitHub tracker pointer. The handoff and review invariant live in CLAUDE.md and spec-review.md instead of inline in the skill. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SdH2gxRtfjoqUWa5hG8SvF
Move the two spec-related glossary entries adjacent and reword Spec/plan review as a process (not a second spec artifact); point the wayfinder cluster at the /wayfinder skill as the authoritative source. Align the spec-review.md title to "Spec/plan review". Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SdH2gxRtfjoqUWa5hG8SvF
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Completes the Matt Pocock engineering-flow adoption.
Notes
Stacked on #686 (needs grilling/domain-modeling + the renamed to-spec/to-tickets); now retargeted to develop. research is a parallel PR (#690) that wayfinder references by name. Supersedes #669.
Test Cases
Checklist
Closes #691
Closes #692